| Conditions | 1 |
| Paths | 1 |
| Total Lines | 116 |
| Code Lines | 98 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 20 | ||
| Bugs | 0 | Features | 0 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | module.exports = function(grunt) { |
||
| 2 | grunt.initConfig({ |
||
| 3 | shell: { |
||
| 4 | rename: { |
||
| 5 | command: |
||
| 6 | 'cp pagantis.zip pagantis-$(git rev-parse --abbrev-ref HEAD).zip \n' |
||
| 7 | }, |
||
| 8 | autoindex: { |
||
| 9 | command: |
||
| 10 | 'php vendor/pagantis/autoindex/index.php . \n' |
||
| 11 | }, |
||
| 12 | composerProd: { |
||
| 13 | command: 'composer install --no-dev' |
||
| 14 | }, |
||
| 15 | composerDev: { |
||
| 16 | command: 'composer install' |
||
| 17 | }, |
||
| 18 | runTestPrestashop17: { |
||
| 19 | command: |
||
| 20 | 'docker-compose down\n' + |
||
| 21 | 'docker-compose up -d selenium\n' + |
||
| 22 | 'docker-compose up -d prestashop17-test\n' + |
||
| 23 | 'echo "Creating the prestashop17-test"\n' + |
||
| 24 | 'sleep 100\n' + |
||
| 25 | 'date\n' + |
||
| 26 | 'docker-compose logs prestashop17-test\n' + |
||
| 27 | 'set -e\n' + |
||
| 28 | 'vendor/bin/phpunit --group prestashop17basic\n' + |
||
| 29 | 'vendor/bin/phpunit --group prestashop17install\n' + |
||
| 30 | 'vendor/bin/phpunit --group prestashop17register\n' + |
||
| 31 | 'vendor/bin/phpunit --group prestashop17buy\n' + |
||
| 32 | 'vendor/bin/phpunit --group prestashop17advanced\n' + |
||
| 33 | 'vendor/bin/phpunit --group prestashop17validate\n' + |
||
| 34 | 'vendor/bin/phpunit --group prestashop17controller\n' |
||
| 35 | }, |
||
| 36 | runTestPrestashop16: { |
||
| 37 | command: |
||
| 38 | 'docker-compose down\n' + |
||
| 39 | 'docker-compose up -d selenium\n' + |
||
| 40 | 'docker-compose up -d prestashop16-test\n' + |
||
| 41 | 'echo "Creating the prestashop16-test"\n' + |
||
| 42 | 'sleep 90\n' + |
||
| 43 | 'date\n' + |
||
| 44 | 'docker-compose logs prestashop16-test\n' + |
||
| 45 | 'set -e\n' + |
||
| 46 | 'vendor/bin/phpunit --group prestashop16basic\n' + |
||
| 47 | 'vendor/bin/phpunit --group prestashop16install\n' + |
||
| 48 | 'vendor/bin/phpunit --group prestashop16register\n' + |
||
| 49 | 'vendor/bin/phpunit --group prestashop16buy\n' + |
||
| 50 | 'vendor/bin/phpunit --group prestashop16advanced\n' + |
||
| 51 | 'vendor/bin/phpunit --group prestashop16validate\n' + |
||
| 52 | 'vendor/bin/phpunit --group prestashop16controller\n' |
||
| 53 | }, |
||
| 54 | runTestPrestashop15: { |
||
| 55 | command: |
||
| 56 | 'docker-compose down\n' + |
||
| 57 | 'docker-compose up -d selenium\n' + |
||
| 58 | 'docker-compose up -d prestashop15-test\n' + |
||
| 59 | 'echo "Creating the prestashop15-test"\n' + |
||
| 60 | 'sleep 90\n' + |
||
| 61 | 'date\n' + |
||
| 62 | 'docker-compose logs prestashop15-test\n' + |
||
| 63 | 'set -e\n' + |
||
| 64 | 'vendor/bin/phpunit --group prestashop15basic\n' + |
||
| 65 | 'vendor/bin/phpunit --group prestashop15install\n' + |
||
| 66 | 'vendor/bin/phpunit --group prestashop15register\n' + |
||
| 67 | 'vendor/bin/phpunit --group prestashop15buy\n' + |
||
| 68 | 'vendor/bin/phpunit --group prestashop15validate\n' + |
||
| 69 | 'vendor/bin/phpunit --group prestashop15controller\n' |
||
| 70 | } |
||
| 71 | }, |
||
| 72 | compress: { |
||
| 73 | main: { |
||
| 74 | options: { |
||
| 75 | archive: 'pagantis.zip' |
||
| 76 | }, |
||
| 77 | files: [ |
||
| 78 | {src: ['controllers/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 79 | {src: ['classes/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 80 | {src: ['docs/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 81 | {src: ['override/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 82 | {src: ['logs/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 83 | {src: ['vendor/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 84 | {src: ['translations/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 85 | {src: ['upgrade/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 86 | {src: ['optionaloverride/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 87 | {src: ['oldoverride/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 88 | {src: ['sql/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 89 | {src: ['lib/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 90 | {src: ['defaultoverride/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 91 | {src: ['views/**'], dest: 'pagantis/', filter: 'isFile'}, |
||
| 92 | {src: 'index.php', dest: 'pagantis/'}, |
||
| 93 | {src: 'pagantis.php', dest: 'pagantis/'}, |
||
| 94 | {src: 'logo.png', dest: 'pagantis/'}, |
||
| 95 | {src: 'logo.gif', dest: 'pagantis/'}, |
||
| 96 | {src: 'LICENSE.md', dest: 'pagantis/'}, |
||
| 97 | {src: 'CONTRIBUTORS.md', dest: 'pagantis/'}, |
||
| 98 | {src: 'README.md', dest: 'pagantis/'} |
||
| 99 | ] |
||
| 100 | } |
||
| 101 | } |
||
| 102 | }); |
||
| 103 | |||
| 104 | grunt.loadNpmTasks('grunt-shell'); |
||
| 105 | grunt.loadNpmTasks('grunt-contrib-compress'); |
||
| 106 | grunt.registerTask('default', [ |
||
| 107 | 'shell:composerDev', |
||
| 108 | 'shell:autoindex', |
||
| 109 | 'shell:composerProd', |
||
| 110 | 'compress', |
||
| 111 | 'shell:composerDev', |
||
| 112 | 'shell:rename' |
||
| 113 | ]); |
||
| 114 | |||
| 115 | //manually run the selenium test: "grunt shell:testPrestashop16" |
||
| 116 | }; |
||
| 117 |